Skip to content

ci(e2e): remove obsolete CocoaPods sample build job#6500

Open
antonis wants to merge 1 commit into
deps/scripts/update-cocoa.shfrom
chore/remove-cocoapods-ci-job
Open

ci(e2e): remove obsolete CocoaPods sample build job#6500
antonis wants to merge 1 commit into
deps/scripts/update-cocoa.shfrom
chore/remove-cocoapods-ci-job

Conversation

@antonis

@antonis antonis commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Removes the dedicated CocoaPods sample build job (sentry-consumption: cocoapods) from sample-application.yml, along with the now single-valued sentry-consumption matrix dimension and its references (job name, SENTRY_USE_XCFRAMEWORK=0 export, archive/upload/log-artifact conditions and names).

Stacked on top of #6409 (Cocoa bump to 9.22.0).

💡 Motivation and Context

sentry-cocoa dropped CocoaPods (trunk) support after 9.19.19.20.0, 9.21.0, and 9.22.0 are published only as GitHub Release xcframeworks and are not on the CocoaPods CDN.

The cocoapods job exports SENTRY_USE_XCFRAMEWORK=0 to exercise the source-build fallback (s.dependency 'Sentry', sentry_cocoa_version), which resolves Sentry from trunk. On #6409 that job fails because Sentry (= 9.22.0) isn't on trunk:

None of your spec sources contain a spec satisfying the dependency: `Sentry (= 9.22.0)`.

The job only passed on main because main is still pinned to 9.19.1. Since the fallback can no longer resolve any Cocoa version past 9.19.1, the regression job is obsolete and blocks every future Cocoa bump.

💚 How did you test it?

  • Validated the workflow YAML parses cleanly.
  • CI on this PR runs the remaining xcframework iOS sample builds.

📝 Checklist

  • I added tests to verify changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • All tests passing.
  • Public API changes reviewed by another Mobile SDK team member or implemented according to the develop docs spec.
  • No breaking changes.

🔮 Next steps

  • The SENTRY_USE_XCFRAMEWORK=0 fallback path in RNSentry.podspec is now effectively dead for versions > 9.19.1. Consider removing it (and the deprecated SENTRY_USE_SPM alias) in a follow-up, unless we intend to keep it for offline/proxy builds pinned to old Cocoa versions.
  • Merge chore(deps): update Cocoa SDK to v9.23.0 #6409 first; this PR should be retargeted to main once its base lands.

@antonis antonis added the ready-to-merge Triggers the full CI test suite label Jul 22, 2026
@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Semver Impact of This PR

None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


  • ci(e2e): remove obsolete CocoaPods sample build job by antonis in #6500
  • chore(deps): update Cocoa SDK to v9.23.0 by github-actions in #6409

🤖 This preview updates automatically when you update the PR.

@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 3858.87 ms 3732.60 ms -126.27 ms
Size 4.98 MiB 6.57 MiB 1.59 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
b0d3373+dirty 3831.75 ms 1227.29 ms -2604.46 ms
ef27341+dirty 3856.13 ms 1231.42 ms -2624.71 ms
9b5615c+dirty 3841.13 ms 1216.33 ms -2624.80 ms
44c8b3f+dirty 3823.85 ms 1207.66 ms -2616.19 ms
68672fc+dirty 3841.58 ms 1228.89 ms -2612.69 ms
4acd4e9+dirty 3847.27 ms 1226.67 ms -2620.60 ms
5748023+dirty 3840.49 ms 1227.43 ms -2613.05 ms
2c735cc+dirty 1229.67 ms 1221.50 ms -8.17 ms
100ce80+dirty 3842.93 ms 1229.52 ms -2613.41 ms
9210ae6+dirty 3815.93 ms 1214.14 ms -2601.79 ms

App size

Revision Plain With Sentry Diff
b0d3373+dirty 5.15 MiB 6.68 MiB 1.53 MiB
ef27341+dirty 5.15 MiB 6.68 MiB 1.53 MiB
9b5615c+dirty 4.98 MiB 6.51 MiB 1.53 MiB
44c8b3f+dirty 5.15 MiB 6.66 MiB 1.51 MiB
68672fc+dirty 5.15 MiB 6.71 MiB 1.55 MiB
4acd4e9+dirty 4.98 MiB 6.53 MiB 1.55 MiB
5748023+dirty 5.15 MiB 6.68 MiB 1.53 MiB
2c735cc+dirty 3.38 MiB 4.74 MiB 1.35 MiB
100ce80+dirty 5.15 MiB 6.67 MiB 1.51 MiB
9210ae6+dirty 5.15 MiB 6.68 MiB 1.53 MiB

@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 430.00 ms 464.33 ms 34.33 ms
Size 49.74 MiB 55.36 MiB 5.61 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
d038a14+dirty 415.60 ms 461.04 ms 45.45 ms
d7d54c6+dirty 458.06 ms 479.94 ms 21.87 ms
68672fc+dirty 425.02 ms 487.56 ms 62.54 ms
ad66da3+dirty 468.46 ms 533.56 ms 65.10 ms
a216cb9+dirty 458.66 ms 531.47 ms 72.81 ms
f170ec3+dirty 428.71 ms 452.18 ms 23.47 ms
e5bb5f6+dirty 430.09 ms 459.96 ms 29.87 ms
41d6254+dirty 424.45 ms 474.34 ms 49.89 ms
68ae91b+dirty 416.44 ms 477.56 ms 61.12 ms
5ee78d6+dirty 551.80 ms 568.27 ms 16.47 ms

App size

Revision Plain With Sentry Diff
d038a14+dirty 48.30 MiB 53.49 MiB 5.19 MiB
d7d54c6+dirty 49.74 MiB 55.09 MiB 5.34 MiB
68672fc+dirty 48.30 MiB 53.61 MiB 5.31 MiB
ad66da3+dirty 48.30 MiB 53.49 MiB 5.19 MiB
a216cb9+dirty 49.74 MiB 55.08 MiB 5.34 MiB
f170ec3+dirty 48.30 MiB 53.57 MiB 5.26 MiB
e5bb5f6+dirty 49.74 MiB 55.09 MiB 5.34 MiB
41d6254+dirty 48.30 MiB 53.60 MiB 5.30 MiB
68ae91b+dirty 49.74 MiB 54.79 MiB 5.05 MiB
5ee78d6+dirty 48.30 MiB 53.58 MiB 5.28 MiB

@antonis
antonis marked this pull request as ready for review July 22, 2026 11:37

@antonis antonis left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The failed E2E tests are related to #6497

@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 3849.58 ms 3742.09 ms -107.50 ms
Size 4.98 MiB 6.57 MiB 1.59 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
b0d3373+dirty 3842.49 ms 1218.49 ms -2624.00 ms
5ee78d6+dirty 3815.71 ms 1217.04 ms -2598.67 ms
9b5615c+dirty 3837.21 ms 1215.33 ms -2621.89 ms
2363c0f+dirty 3834.61 ms 1224.46 ms -2610.15 ms
7436d0f+dirty 3861.51 ms 1231.07 ms -2630.44 ms
4acd4e9+dirty 3806.73 ms 1208.48 ms -2598.25 ms
0307fa4+dirty 3853.00 ms 1224.67 ms -2628.33 ms
3a829f0+dirty 3822.15 ms 1211.04 ms -2611.10 ms
bf168a4+dirty 3846.83 ms 1221.33 ms -2625.49 ms
57e0069+dirty 3842.23 ms 1210.00 ms -2632.23 ms

App size

Revision Plain With Sentry Diff
b0d3373+dirty 5.15 MiB 6.68 MiB 1.53 MiB
5ee78d6+dirty 5.15 MiB 6.69 MiB 1.53 MiB
9b5615c+dirty 4.98 MiB 6.51 MiB 1.53 MiB
2363c0f+dirty 4.98 MiB 6.46 MiB 1.49 MiB
7436d0f+dirty 5.15 MiB 6.70 MiB 1.54 MiB
4acd4e9+dirty 4.98 MiB 6.53 MiB 1.55 MiB
0307fa4+dirty 4.98 MiB 6.46 MiB 1.48 MiB
3a829f0+dirty 5.15 MiB 6.70 MiB 1.54 MiB
bf168a4+dirty 4.98 MiB 6.47 MiB 1.49 MiB
57e0069+dirty 4.98 MiB 6.50 MiB 1.52 MiB

@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 420.72 ms 461.53 ms 40.81 ms
Size 49.74 MiB 55.36 MiB 5.61 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
3a829f0+dirty 410.78 ms 454.22 ms 43.44 ms
bfba737+dirty 425.19 ms 465.98 ms 40.79 ms
37a2091+dirty 429.71 ms 477.00 ms 47.29 ms
7ac3378+dirty 410.67 ms 442.60 ms 31.92 ms
3b6e9f9+dirty 442.39 ms 486.44 ms 44.05 ms
44c8b3f+dirty 492.13 ms 563.47 ms 71.34 ms
21a1e70+dirty 437.55 ms 507.10 ms 69.56 ms
c151573+dirty 485.39 ms 495.18 ms 9.79 ms
580fb5c+dirty 515.72 ms 585.38 ms 69.66 ms
774257e+dirty 541.42 ms 575.55 ms 34.13 ms

App size

Revision Plain With Sentry Diff
3a829f0+dirty 48.30 MiB 53.58 MiB 5.28 MiB
bfba737+dirty 49.74 MiB 55.09 MiB 5.34 MiB
37a2091+dirty 48.30 MiB 53.58 MiB 5.28 MiB
7ac3378+dirty 43.94 MiB 48.99 MiB 5.05 MiB
3b6e9f9+dirty 48.30 MiB 53.54 MiB 5.23 MiB
44c8b3f+dirty 48.30 MiB 53.46 MiB 5.15 MiB
21a1e70+dirty 49.74 MiB 54.82 MiB 5.07 MiB
c151573+dirty 48.30 MiB 53.54 MiB 5.24 MiB
580fb5c+dirty 49.74 MiB 54.79 MiB 5.05 MiB
774257e+dirty 48.30 MiB 53.58 MiB 5.28 MiB

@bruno-garcia
bruno-garcia force-pushed the deps/scripts/update-cocoa.sh branch 2 times, most recently from 63e7c97 to 932a11b Compare July 23, 2026 04:02
@antonis
antonis force-pushed the chore/remove-cocoapods-ci-job branch from 139f852 to 6e9ff1b Compare July 23, 2026 07:33
@bruno-garcia
bruno-garcia force-pushed the deps/scripts/update-cocoa.sh branch from 932a11b to fc8bd37 Compare July 23, 2026 08:08
@bruno-garcia
bruno-garcia force-pushed the deps/scripts/update-cocoa.sh branch 3 times, most recently from eca25be to 88771ff Compare July 23, 2026 14:54
sentry-cocoa dropped CocoaPods (trunk) support after 9.19.1, so the
source-build fallback (SENTRY_USE_XCFRAMEWORK=0) can no longer resolve
newer versions. The dedicated CocoaPods regression job now fails on any
Cocoa bump past 9.19.1, so remove it and the now single-valued
sentry-consumption matrix dimension.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@antonis
antonis force-pushed the chore/remove-cocoapods-ci-job branch from d0887e0 to 650469e Compare July 23, 2026 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Triggers the full CI test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants